[NAS Backup] Suppress Errors in Disk Usage Calculation that Caused Backup to Fail.#13424
[NAS Backup] Suppress Errors in Disk Usage Calculation that Caused Backup to Fail.#13424daviftorres wants to merge 15 commits into
Conversation
Pulling upstream.
Handle potential errors when calculating disk usage.
|
This is the equivalent command for applying the fix: We haven't confirmed the exact root cause of the So, I am running tests with |
Add timeout for unmounting backup mount point and cleanup.
Proposed Changes Rationalebackup_size=$(du -sb "$dest" 2>/dev/null | cut -f1) || true
timeout 60 umount "$mount_point" 2>/dev/null || true
rmdir "$mount_point" 2>/dev/null || true
echo -n "$backup_size"
|
|
Dear @abh1sar , do you think you can help me with this bug? Regards, |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #13424 +/- ##
=========================================
Coverage 18.88% 18.89%
- Complexity 18223 18226 +3
=========================================
Files 6174 6174
Lines 555226 555226
Branches 67774 67774
=========================================
+ Hits 104872 104895 +23
+ Misses 438834 438810 -24
- Partials 11520 11521 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR adjusts the KVM NAS backup script’s “statistics/cleanup” section so that failures while computing backup disk usage (and related cleanup commands) don’t cause an otherwise successful backup job to be marked as failed.
Changes:
- Capture
duoutput intobackup_sizeand suppressdustderr to avoid failing the script during size calculation. - Add
timeoutaroundumountand suppress errors fromumount/rmdir. - Emit the computed backup size at the end of
backup_running_vm().
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| backup_size=$(du -sb "$dest" 2>/dev/null | cut -f1) || true | ||
|
|
||
| timeout 60 umount "$mount_point" 2>/dev/null || true | ||
| rmdir "$mount_point" 2>/dev/null || true | ||
|
|
Description
This PR tried to prevent the failure of the job at the statistics section of a backup that has actually succeeded.
Apparently, it also fixes some silent failures I previously reported in #11727
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?